Revised the conditional operator article #9125
Conversation
There was a problem hiding this comment.
Thanks, @pkulikov. Your changes look really good. I've left a few comments and suggestions. But the final example isn't being included in the build, and I have no idea why. I'll close and reopen this PR to see if it mysteriously appears with the rebuild.
docs/csharp/language-reference/operators/conditional-operator.md
Outdated
Show resolved
Hide resolved
docs/csharp/language-reference/operators/conditional-operator.md
Outdated
Show resolved
Hide resolved
docs/csharp/language-reference/operators/conditional-operator.md
Outdated
Show resolved
Hide resolved
docs/csharp/language-reference/operators/conditional-operator.md
Outdated
Show resolved
Hide resolved
docs/csharp/language-reference/operators/conditional-operator.md
Outdated
Show resolved
Hide resolved
|
|
||
| Usage of the conditional operator over an [if-else](../keywords/if-else.md) statement might result in more concise code in cases when you need conditionally to compute a value. The following example demonstrates two ways to classify an integer as negative or nonnegative: | ||
|
|
||
| [!code-csharp[conditional and if-else](~/samples/snippets/csharp/language-reference/operators/ConditionalExamples.cs#CompareWithIf)] |
There was a problem hiding this comment.
The example is not building, and I have no idea why:
The build is also not generating any warnings.
There was a problem hiding this comment.
@rpetrusha I also didn't notice any warnings even before the samples PR was merged. Does it build properly after merging the samples PR?
There was a problem hiding this comment.
The latest build shows no warnings, and the code appears as expected, @pkulikov
I have no idea of why one block of code was not found when the others were.
|
Closing and reopening to begin new build. |
|
@rpetrusha thanks for review; I've addressed your feedback. |
3 participants
refkeyword article